Skip to content

86 replace query parameter navigation and improve navbar #88

Jump to bottom
Merged
merged 11 commits into from
Nov 12, 2026
Merged

86 replace query parameter navigation and improve navbar #88

merged 11 commits into from
Nov 12, 2026
+242 −242

Conversation

krgjert
Copy link
Member

@krgjert krgjert commented Nov 9, 2026

Purpose

Replace query-parameter navigation and simplify the header.

Changes

  • Added React Router for navigation.
  • Updated links and category routes.
  • Added a mobile hamburger menu.
  • Improved search and active-link styling.
  • Organized components into folders.
  • Updated the favicon and page title.

Related issues

Refs #86.

Verification

  • Build and lint passed.
  • All 12 tests passed.
  • Manual browser testing remains.

Screenshots

To be added.

Checklist

  • I have reviewed my own changes.
  • The changes meet the linked issue's acceptance criteria.
  • I have run checks and recorded the results.
  • I have added or updated tests where needed.
  • I have updated documentation where needed.
  • I have included screenshots.
Screenshot 2026-11-09 at 19 43 55 Screenshot 2026-11-09 at 19 44 20 Screenshot 2026-11-09 at 19 44 32

Kristian Gjertsen added 10 commits November 9, 2026 18:17
Renamed file NavBar to Header
a6bbbf5
Implemented react-router for navigation instead og query- parametre
5535290
Restructured NavBar, Footer, and button components in folders
a2c60ca
Refactor navigation to use react-router, update component imports, an…
12db7b0 
…d enhance UI elements
Refactor navigation to use react-router in multiple components, repla…
3b973eb 
…cing query parameters with Link and useNavigate for improved routing
Refactor focus-visible, encapsulating them within a base layer, and r…
5401eac 
…ounded the corner
Refactor NavBar component and integrating responsive menu
962a3d4
Update document title to 'Full Circuit'
943a1b8
Replace favicon.svg with favicon.ico and remove unused icon files
e82ece1
Add shadow to NavBar for improved visual hierarchy from background el…
241c88f 
…ements
@krgjert krgjert requested a review from a team as a code owner November 9, 2026 18:47
@krgjert krgjert self-assigned this Nov 9, 2026
@krgjert krgjert added the enhancement New feature or request label Nov 9, 2026
@krgjert krgjert added this to the 4. underveisinnlevering milestone Nov 9, 2026
wilhelsr
wilhelsr approved these changes Nov 12, 2026
View reviewed changes
Copy link
Member

@wilhelsr wilhelsr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Nice improvements, and all checks are passing. Approved!

@wilhelsr wilhelsr merged commit 46ac326 into dev Nov 12, 2026
@krgjert krgjert deleted the 86-replace-query-parameter-navigation-and-improve-navbar branch November 14, 2026 09:30
Sign in to join this conversation on GitHub.
Reviewers

@wilhelsr wilhelsr wilhelsr approved these changes

Assignees

@krgjert krgjert

Labels
enhancement New feature or request
Projects
None yet
Milestone
4. underveisinnlevering
Development

Successfully merging this pull request may close these issues.
2 participants
@krgjert @wilhelsr